Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Use clang-15 for paritytech/base-ci-linux #490

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Use clang-15 for paritytech/base-ci-linux #490

merged 1 commit into from
Nov 24, 2022

Conversation

rcny
Copy link
Contributor

@rcny rcny commented Nov 24, 2022

clang-14 is still too low for the CI workloads even there were reports that it fixes related linker problems locally. Bumping further to clang-15.

cc @muharem @samelamin

@rcny rcny requested a review from a team November 24, 2022 14:06
@rcny rcny self-assigned this Nov 24, 2022
@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

Supersedes #488.

@rcny rcny merged commit c27cc84 into master Nov 24, 2022
@rcny rcny deleted the vi-clang-15 branch November 24, 2022 14:08
@samelamin
Copy link

Is it safe to rerun workloads now?

@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

Not yet, we're rebuilding the images, I'll post an update here when the we'll push updated images.

@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

So, looks like it's not related to the clang version as we continue to observe the same behavior with the same linker error even with clang-15. I will investigate more.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants